Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add ci/cd for github actions #1070

Merged
merged 34 commits into from
May 5, 2022
Merged

add ci/cd for github actions #1070

merged 34 commits into from
May 5, 2022

Conversation

joamatab
Copy link
Contributor

add ci/cd for uploading pypi wheels for macOs, windows and Linux

@thomaslima
Copy link
Collaborator

Dear @joamatab ,

I have reworked this PR to use cibuildwheel, as mentioned in #1069 . Can you review and merge my commits from https://github.com/thomaslima/klayout/tree/issue-1070 into your branch? There's a single workflow file now that does all the ubuntu manylinux/musllinux builds plus macOS builds (with cache between builds). The advantage is that cibuildwheel is maintained by pypa so it will take little work to update to future python versions.

@klayoutmatthias I have already created a pypi_password and test_pypi_password secret token. Right now, it is configured to upload to test pypi at every commit. Please uncomment lines 82 and 96 and comment line 94 in build.yml before/after merge to change the target repository to the real pypi one.

Thanks,
Thomas

@joamatab
Copy link
Contributor Author

joamatab commented May 5, 2022

Awesome! thank you Thomas, I merged your commits

@klayoutmatthias klayoutmatthias merged commit 5203f11 into KLayout:master May 5, 2022
klayoutmatthias pushed a commit that referenced this pull request May 5, 2022
* add ci/cd for github actions

* fix workflow and add workflow_dispatch

* remove dist

* fix

* add gcc

* sudo apt install build essential

* add curl

* install curl

* build essential

* update

* test code on mac os

* release on tagged version

* better names

* fix numbers for 3.10

* add cancel workflow for tests

* fix test macOs

* Adding ccache to github action

* creating dependency between jobs

* using cibuildwheel

* ccache path bug

* Testing after building wheels

* misunderstood cibuildwheel

* fixing error in manylinux build

* fixing error in manylinux build (2)

* disabling fail fast (temporary)

* Don't repair macOS wheels

* Not building pypy wheels

* disabling 32-bit linux wheels

* macos-latest only (prevent duplicate)

* fixing musllinux

* fixing ccache

* fixing cache persistence

* adding deploy job

* last debug

Co-authored-by: Thomas Ferreira de Lima <[email protected]>
klayoutmatthias pushed a commit that referenced this pull request May 11, 2022
* add ci/cd for github actions

* fix workflow and add workflow_dispatch

* remove dist

* fix

* add gcc

* sudo apt install build essential

* add curl

* install curl

* build essential

* update

* test code on mac os

* release on tagged version

* better names

* fix numbers for 3.10

* add cancel workflow for tests

* fix test macOs

* Adding ccache to github action

* creating dependency between jobs

* using cibuildwheel

* ccache path bug

* Testing after building wheels

* misunderstood cibuildwheel

* fixing error in manylinux build

* fixing error in manylinux build (2)

* disabling fail fast (temporary)

* Don't repair macOS wheels

* Not building pypy wheels

* disabling 32-bit linux wheels

* macos-latest only (prevent duplicate)

* fixing musllinux

* fixing ccache

* fixing cache persistence

* adding deploy job

* last debug

Co-authored-by: Thomas Ferreira de Lima <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants